-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
git: re-add support for git decoration preferences #11674
Conversation
The commit re-adds support for `git.decorations.enabled` and `git.decorations.colors` which were previously not referenced and supported after a change in `v1.14.0`. Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can confirm that the changes work as expected. I have some comments though, see below.
Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
@msujew thank you for your help, I like it better than having to cache the event itself. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good to me 👍
- Decorations are applied/colored correctly when both preferences are enabled
- Decorations can be completely disabled
- Decoration colors can be disabled independently of that
- Changes these preferences is applied correctly and instantly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have verified the changes as per instructions and with a different theme and it works as expected ! 👍
The code looks good to me as well !!
The commit re-adds support for `git.decorations.enabled` and `git.decorations.colors` which were previously not referenced and supported after a change in `v1.14.0`. Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com> Co-authored-by: Mark Sujew <mark.sujew@typefox.io>
The commit re-adds support for `git.decorations.enabled` and `git.decorations.colors` which were previously not referenced and supported after a change in `v1.14.0`. Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com> Co-authored-by: Mark Sujew <mark.sujew@typefox.io>
What it does
The pull-request re-adds support for
git.decorations.enabled
andgit.decorations.colors
which were previously not referenced and supported after a change in #8911 (v1.14.0).git-decorations.mov
How to test
git decorations
git.decorations.enabled
hides/shows decorations in the navigator for the resourcesgit.decorations.colors
hides/displays colors for the decorations in the navigator for the resourcesReview checklist
Reminder for reviewers
Signed-off-by: vince-fugnitto vincent.fugnitto@ericsson.com